This refactor applies the VR-divergence guideline by replacing direct REL::Module::IsVR() runtime checks with the cached globals::game::isVR where call timing is safe. Remaining raw module checks are intentionally limited to init/early-init and header initialization contexts.

• Scope: timing-safe runtime callsites

  • Swapped direct module checks to globals::game::isVR across major runtime paths in src/ (deferred rendering, hooks, shader cache runtime logic, UI/game utils, feature runtime code including LLF/SSGI/VR/upscaling).
  • Goal: use one cached runtime source instead of repeated module queries during normal execution.

• Intentional raw-check exceptions

  • Kept raw REL::Module::IsVR() at the cache population site in Globals.cpp.
  • Kept raw checks in early plugin load path (XSEPlugin.cpp) where globals::game::isVR is not yet initialized.
  • Kept raw checks in header/default-init/macro/static-init contexts (Globals.h, ShaderCache.h, Utils/Game.h, ScreenSpaceGI.h, LightLimitFix.h, Menu.h) where initialization order/timing can precede ReInit.
  • Added brief one-line reasons at non-obvious retained sites.

• Net effect

  • Runtime VR branching is now predominantly routed through globals::game::isVR, with raw calls constrained to legitimate init-order exceptions.

// Before
if (REL::Module::IsVR()) {
    // VR path
}

// After (timing-safe runtime path)
if (globals::game::isVR) {
    // VR path
}